Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

improve all probes for all components #97

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

improve all probes for all components #97

wants to merge 2 commits into from

Conversation

edevosc2c
Copy link
Member

@edevosc2c edevosc2c commented Jul 4, 2024 •
edited
Loading

  • Switch container port name from http-proxy to http: http-proxy word is confusing, it's not an HTTP proxy.
  • Add healthcheck probes to components that didn't have it.
  • Stop using tcpCheck and instead use httpCheck for being 100% sure that the traffic can flow to the application.
  • When possible use the named container port for things like liveness, service and more. This allows to specify the application number port only once.
  • Remove case for old Kubernetes version in default ingress

TODO:

  • Need to make sure that all the probes won't affect the startup of all apps.
  • allow customization of liveness and startup probes

jeanpommier
jeanpommier approved these changes Jul 4, 2024
View reviewed changes
Copy link
Member

@jeanpommier jeanpommier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice job !

@edevosc2c edevosc2c marked this pull request as draft July 5, 2024 09:44
@edevosc2c edevosc2c added the enhancement New feature or request label Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeanpommier jeanpommier jeanpommier approved these changes

@pmauduit pmauduit Awaiting requested review from pmauduit

@jeanmi151 jeanmi151 Awaiting requested review from jeanmi151

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@edevosc2c @jeanpommier